Conversation
WalkthroughUpdates build and tooling versions (Paperweight plugin and Minecraft), switches Nix to use gradle_8 with JDK override, adjusts GameProfile property access in ItemBuilder, and removes the publishing repositories block from Gradle while keeping publications. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Poem
Pre-merge checks and finishing touches❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✨ Finishing touches
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Note Docstrings generation - SUCCESS |
Docstrings generation was requested by @JonasFranke. * #234 (comment) The following files were modified: * `src/main/java/de/j/stationofdoom/util/ItemBuilder.java`
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (1)
build.gradle.kts (1)
94-94: Consider updating the API version to match the Minecraft version.The
apiVersionis set to "1.20", but the Minecraft version has been bumped to 1.21.10. Consider updating this to "1.21" to align with the target Minecraft version and ensure compatibility with Paper's API.Apply this diff:
- apiVersion = "1.20" + apiVersion = "1.21"
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (3)
build.gradle.kts(1 hunks)flake.nix(1 hunks)src/main/java/de/j/stationofdoom/util/ItemBuilder.java(2 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)
- GitHub Check: build
- GitHub Check: paths-filter
- GitHub Check: build
🔇 Additional comments (4)
build.gradle.kts (1)
12-12: LGTM!The Minecraft version bump to 1.21.10 aligns with the PR objective.
src/main/java/de/j/stationofdoom/util/ItemBuilder.java (2)
92-92: LGTM!The API change from
getProperties()toproperties()correctly reflects the updated GameProfile API in Paper 1.21.10.
113-113: LGTM!The API change from
getProperties()toproperties()is applied consistently with the change at Line 92.flake.nix (1)
13-13: LGTM!Switching to
gradle_8aligns with the Paperweight 2.0.0-beta.18 upgrade, which requires Gradle 8.
| id("xyz.jpenilla.run-paper") version "2.3.1" | ||
| id("net.minecrell.plugin-yml.bukkit") version "0.6.0" | ||
| id("io.papermc.paperweight.userdev") version "1.7.7" | ||
| id("io.papermc.paperweight.userdev") version "2.0.0-beta.18" |
There was a problem hiding this comment.
🧩 Analysis chain
Verify the stability of the beta plugin version.
The Paperweight Userdev plugin version 2.0.0-beta.18 is a beta release. Please verify that this version is stable and suitable for production use, or document any known issues.
🌐 Web query:
Is Paperweight Userdev 2.0.0-beta.18 stable for production use?
💡 Result:
Short answer: No — 2.0.0-beta.18 is a beta pre-release and not intended as a guaranteed production-stable release. Use a stable release (or thoroughly test the beta in your environment) and follow PaperMC guidance to prefer the latest supported non-beta release. [1][2][3]
Sources:
- paperweight-userdev docs (shows plugin is maintained and advises using latest supported version). [1]
- paperweight GitHub release for v2.0.0-beta.18 (release is labeled beta). [2]
- open issue showing known incompatibilities/bugs (example: configuration-cache incompatibility). [3]
Use a stable Paperweight Userdev release
Paperweight Userdev 2.0.0-beta.18 is a pre-release and not guaranteed production-stable. Switch to the latest non-beta version or thoroughly test this beta in your environment and document any known issues.
🤖 Prompt for AI Agents
In build.gradle.kts around line 7 the plugin version is set to the pre-release
io.papermc.paperweight.userdev version "2.0.0-beta.18"; replace this with the
latest stable non-beta release (check the Paperweight/Userdev release notes or
Gradle Plugin Portal for the current stable version) or, if you must keep the
beta, add a clear comment above the line documenting that this is a beta, list
any known issues, and add a CI/job to run thorough integration tests validating
the build against your target Minecraft/Paper versions so the risk is tracked.
Paper 1.21.10
Summary by CodeRabbit
Chores
Refactor